Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
There was a problem hiding this comment.
Pull request overview
This PR addresses issue #110 by ensuring file nodes originating from .clayer.yml carry sufficient context (layerUri) for layer-specific actions (e.g., removing a file entry from the layer YAML) in the Solution Outline tree.
Changes:
- Extend context-menu attribute assignment to optionally set
layerUriwhen items come from a layer (topTag === 'layer'). - Thread the YAML “top tag” context from group construction into file-node construction.
- Add a
topTagfield toFileItemBuilderso file nodes can setlayerUriwhen appropriate.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/views/solution-outline/tree-structure/solution-outline-utils.ts |
Updates setContextMenuAttributes to accept topTag and set layerUri for layer-derived items. |
src/views/solution-outline/tree-structure/solution-outline-project-items.ts |
Computes/passes topTag through group creation so file nodes can be tagged as layer vs project. |
src/views/solution-outline/tree-structure/solution-outline-file-item.ts |
Adds topTag to FileItemBuilder and passes it into setContextMenuAttributes for file nodes. |
…ls.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Coverage Impact This PR will not change total coverage. Modified Files with Diff Coverage (3)
🛟 Help
|
Fixes
Changes
Screenshots
Checklist